Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reprostim-*capture -- start adding tests via ctests framework #78

Merged
merged 17 commits into from
Feb 27, 2024

Conversation

vmdocua
Copy link
Collaborator

@vmdocua vmdocua commented Feb 20, 2024

Setup basic structure for CTest, create test placeholder in each project, start creating some tests. Closes #74 👍

  • Refine project directory structure to better fit CTest( in app projects move code under src; keep config at root level; move public capturelib to reprostim namespace/folder; create test folder in each subproject dedicated to test, and keep each test closer to project and source code), so it will look close to this picture:
    project_structure
  • Start implementing simple CTest with CMake (e.g. run --help for reprostim-*capture utilities).
  • Research and create simple unit test starter examples in capturelib for few APIs, consider catch2 or gtest as start point.
  • Integrate tests with github workflow.
  • Add notes in Capture/README.md regarding testing with CTest.

@vmdocua vmdocua self-assigned this Feb 20, 2024
@vmdocua vmdocua marked this pull request as ready for review February 27, 2024 14:18
@vmdocua vmdocua merged commit 97a1ab2 into master Feb 27, 2024
1 check passed
@yarikoptic yarikoptic deleted the bf-capture-issue74 branch January 22, 2025 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

reprostim-*capture -- start adding tests via ctests framework
1 participant